Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Add prettier #1640

Merged
merged 3 commits into from
Oct 20, 2023
Merged

Add prettier #1640

merged 3 commits into from
Oct 20, 2023

Conversation

vardan10
Copy link
Contributor

@vardan10 vardan10 commented May 16, 2023 •
edited
Loading

What was the problem?

This PR resolves #1185

How was it solved?

  • Add prettier
  • Format code

How was it tested?

  • By running tests
  • Manually

@vardan10 vardan10 changed the base branch from development to release/0.7.0 May 16, 2023 09:41
@vardan10 vardan10 self-assigned this May 16, 2023
@vardan10 vardan10 force-pushed the 1185-add-prettier branch from d5d1bad to 1864367 Compare May 16, 2023 11:04
github-advanced-security[bot]
github-advanced-security bot found potential problems Aug 25, 2023
View reviewed changes
services/blockchain-connector/shared/sdk/index.js Dismissed Show dismissed Hide dismissed
services/blockchain-indexer/shared/indexer/genesisBlock.js Dismissed Show dismissed Hide dismissed
services/blockchain-indexer/shared/indexer/indexStatus.js Dismissed Show dismissed Hide dismissed
@codecov
Copy link

codecov bot commented Aug 25, 2023 •
edited
Loading

Codecov Report

Merging #1640 (f56c56f) into release/0.7.0 (789d641) will decrease coverage by 0.24%.
The diff coverage is 52.33%.

Impacted file tree graph

@@                Coverage Diff                @@
##           release/0.7.0    #1640      +/-   ##
=================================================
- Coverage          66.84%   66.60%   -0.24%     
=================================================
  Files                298      298              
  Lines               5128     5214      +86     
  Branches             925      935      +10     
=================================================
+ Hits                3428     3473      +45     
- Misses              1700     1741      +41
Files Coverage Δ
services/blockchain-app-registry/config.js 100.00% <100.00%> (ø)
...ices/blockchain-app-registry/shared/utils/param.js 100.00% <100.00%> (ø)
...app-registry/tests/constants/downloadRepository.js 100.00% <ø> (ø)
...hain-app-registry/tests/constants/metadataIndex.js 100.00% <ø> (ø)
services/blockchain-connector/config.js 100.00% <100.00%> (ø)
services/blockchain-connector/shared/utils/fs.js 100.00% <100.00%> (ø)
...ces/blockchain-connector/tests/constants/parser.js 100.00% <ø> (ø)
services/blockchain-indexer/config.js 100.00% <100.00%> (ø)
...aService/business/dynamicReward/annualInflation.js 50.00% <100.00%> (ø)
...shared/dataService/business/dynamicReward/index.js 100.00% <100.00%> (ø)
... and 69 more

... and 66 files with indirect coverage changes

@vardan10 vardan10 requested review from sameersubudhi and priojeetpriyom and removed request for sameersubudhi August 25, 2023 13:14
@vardan10 vardan10 marked this pull request as draft August 25, 2023 13:27
@vardan10 vardan10 marked this pull request as ready for review October 20, 2023 12:34
@sameersubudhi sameersubudhi merged commit afceb5f into release/0.7.0 Oct 20, 2023
6 of 8 checks passed
@sameersubudhi sameersubudhi deleted the 1185-add-prettier branch October 20, 2023 14:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@sameersubudhi sameersubudhi sameersubudhi approved these changes

@priojeetpriyom priojeetpriyom priojeetpriyom approved these changes

Assignees

@vardan10 vardan10

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add prettier
3 participants
@vardan10 @priojeetpriyom @sameersubudhi